starknet_transaction_prover,blockifier_reexecution: drop starknet-sierra-compile sidecar from docker images#14418
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
PR SummaryLow Risk Overview
Reviewed by Cursor Bugbot for commit 0f3d0b3. Bugbot is set up for automated code reviews on this repo. Configure here. |
…le sidecar from docker images Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
4a5c598 to
0f3d0b3
Compare

Why
Follow-up to #14406, which made
blockifier_reexecutioncompile Sierra→Casm in-process. Both Docker images that build on it shipped thestarknet-sierra-compilesidecar binary purely for that path; it is now dead weight.What
crates/starknet_transaction_prover/Dockerfile: the prover is built--features stwo_proving(cairo_native disabled), so Sierra→Casm was its only compiler dependency. Removed theinstall_compiler_binaries.sh --sierrastep, theCARGO_TOOLS_ROOTenv, and the COPY of the tools tree. The runtime image is now a single self-contained binary — no compiler sidecar at all.crates/blockifier_reexecution/replay/Dockerfile: built--features cairo_native, so it still needsstarknet-native-compile(the compare-native path, unaffected by blockifier_reexecution: compile Sierra to Casm in-process via library call #14406). Switched the install from both compilers to--nativeonly; kept the tools COPY and thebinutils/libc6-devruntime deps thatstarknet-native-compilerequires. Comments updated accordingly.Validation
docker build --checkon both Dockerfiles — "Check complete, no warnings found."scripts/install_compiler_binaries.sh --nativeis a supported standalone invocation; the replay base stage installs LLVM 19, so itsllvm-config-19gate passes.starknet_transaction_prover_ci.ymldocker-build;blockifier_reexecution_docker_publish.yml), giving a full end-to-end build.Stacked on #14406.
🤖 Generated with Claude Code